Undefine GTK_DISABLE_DEPRECATED in testinput to keep it compiling

Deprecate GtkInputDialog

Deprecate the widget and adjust the documentation.

https://bugzilla.gnome.org/show_bug.cgi?id=594903

Fix build on Windows

Use G_BEGIN/END_DECLS in gtkcellrendererspinner.h

Fix thinkos in which fields get used

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Oops, it's gsize, not GSize

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Fix a g_strdup_printf()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Put the list of column types in a macro

Every call to _gtk_file_system_model_new*() gets the same list of column types,
so we now keep the list in a macro instead of having duplicated copies all around.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify the arguments of g_file_query_info_async()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make copy_attribute() a function instead of a macro

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Don't leak a treepath if the treeview is not populated yet

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Factor out function to set a fixed size for the icon cell renderers

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Comment that the column order needs to be kept in sync across two functions

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Fix typo in a comment

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make get_selected_file() return a ref'ed file

The old semantics was to return a GFile* owned by the file system model; the new
semantics is to hand out new references whenever possible.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

In set_filter(), handle the case where the new filter is the same as the old filter

Signed-off-by: Federico Mena Quintero <federico@novell.com>

s/get_is_visible/iter_is_visible for clarity

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Remove obsolete comment

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make the code match the docs in _gtk_file_system_model_clear_cache()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Free some missing fields in ::finalize()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify variable names in gtk_file_system_model_sort()

To comply with the i -> indexes; r -> rows convention.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Nothing to do for ref/unref_node

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify a couple of spots with comments

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify array indexes vs. row numbers

There was some confusion between "index" as used for the model->files[] array,
and node->index as used for our 1-based row numbers.  Now we use "index" only
for indices in the model->files[] array, and node->row for row numbers.  Functions
and variables are renamed to clarify whether they refer to indexes or rows.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Big comment on how GtkFileSystemModel works

Let's have some documentation on the idea behind this beast...

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Remove obsolete code to use a cached mime-type

The non-standard "filechooser::mime-type" was a remnant of the recent-files code using
a hand-built GFileInfo; now we just query the file info ourselves.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Fix thinko in node_should_be_visible()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Factory out functions to emit row_inserted, row_changed, row_deleted

This is to avoid temporary variables for the path/iter and to avoid duplicated code.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

When updating a file, handle the case where the old file info is the same as the new one

This prevents us from inadvertently losing the last ref to the info.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

When removing a file, also remove it from the file_lookup hash table

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Comment on how the file_lookup hash table gets rebuilt on demand

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify the arguments to g_file_query_info_async()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Show an error dialog when we can't read the folder's contents

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify the code flow in gtk_file_system_model_got_files()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Remove the dir_thaw_source while disposing the model

Otherwise that timeout may trigger after the model has died.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make _gtk_file_system_model_remove_file() static

It was only used internally by the model.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Make _gtk_file_system_model_add_file() static

It was only used internally by the model, anyway.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify the comments that some fields in GtkFileSystemModel can be NULL

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify the arguments to g_file_monitor_directory()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Clarify an argument to g_file_enumerate_children_async()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Fix and tighten an assertion

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Complete the documentation string for _gtk_file_system_model_new()

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Start with a file array with a resonable preallocated size

g_array_new() doesn't reserve any size by default, so during the initial population
of the file array, we'll do more reallocs than strictly needed.  We'll start with
a reasonable preallocated size, in this case the number of files which we can
get in a single chunk out of GIO.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Compute the node size only once to avoid the scary macro

Signed-off-by: Federico Mena Quintero <federico@novell.com>

g_error() as soon as we catch an invalid column type

There's no point in running a GtkFileSystemModel with invalid column types.
This way we can also avoid clearing the memory of the column_types array.

Signed-off-by: Federico Mena Quintero <federico@novell.com>

Improve show_and_select_files() function

The previous function enumerated the whole directory and used a lot of
outdated API to decide how to show files.
The new code queries the filesystem model to decide about this.
The now unused old functions were removed.

Remove unused error argument from show_and_select_files()

Enable the size column in recent files/search

Previously information about file sizes was not available for search
results and recent files, so the column was always hidden. As this
information is now available, we can stop the special handling and use
the same setting as in browse mode.

Use the faster accessor function in the sort functions

Use the faster _gtk_file_system_model_get_value() function instead of
gtk_tree_model_get() inside the sort functions. This gives a significant
speed-up when sorting large lists.
In a test case with 40.000 files, the sorting time went from ~5 seconds
to less than 0.5 seconds for my test case. There is 2 significant
problems with gtk_tree_model_get() that cause this:
1) The value is copied, which takes quite a bit of time for strings.
   ~25% of excessive time or ~1 second in my test
2) The tree model functions need to lookup the interface vfunc. And
   gtk_tree_model_get() doesn't do that only once, but multiple times
   (verifying column id, getting the actual value, ...)
   ~75% of excessive time or ~3 seconds in my test

Convert recent files to a GtkFileSystemModel

This does to the recent files what the last commit did to the search. It
uses a GtkFileSystemModel and its API niceties to show the recent files.

Convert search to use a GtkFileSystemModel

Replace the list model code with the file system model and use all the
file system model API niceties we get from that.
Also adds the function _gtk_file_system_model_add_and_query_file() which
g_file_query_info()'s the file before adding it, so it gets added with
the right information.

Don't count the "enter directory name" line as a selected file

Add a constructor to filesystem model that does not monitor a directory

This is in preparation for switching search and recent models to use
GtkFileSystemModel

Make the filesystem model filter API use a GtkFileFilter

This gets rid of the vfunc API and does exactly what the file chooser
wants.

Only query the absolutely necessary attributes

Since the time taken by g_file_enumerate_children() depends a lot on the
attributes that are queried, we query the minimum attributes that need
to be queired to display the file chooser.

In particular, the attributes for loading the icon are ignored, as icons
are loaded on demand (see previous commit).

Load icons on-demand

Because loading icons takes a noticable performance, this code loads the
pixbuf on demand and only loads icons for rows that are visible. There
is a few caveats to this:
- The pixbuf cell renderer must report the proer size even if the icon
  is not yet loaded. This is achieved by setting a fixed size.
- On theme changes the cahced pixbufs and the cell renderer must be
  updated to conform to the new theme.

Use the GtkTreeSortable of the filesystem model

Previously, there was a GtkTreeSortModel wrapped around the filesystem
model to make it sortable. As the new implementation implements the
GtkTreeSortable interface, we can use this instead.

Clean up code to use the model directly

A lot of code special cases accesses to the tree view for the different
browse modes, which was previously necessary, because the models were
different. Now that they are identical in the first columns, there is no
such need anymore, and the functions don't need to be special cased.

Replace the cell data computations

Previously custom functions were used to compute the data passed to the
cell renderers. Now that all this data is saved by the tree models with
compatible nodes, the usual default attribute-to-column mapping can be
used.
With this, caching of the values can happen in the tree model, which
avoids costly lookups of icons or computation of strings. Last but not
least it avoids spurious bugs that could happen when strings changed
without anyone noticing, like the mtime when a new day begins.

Consolidate tree models for different browse modes

All tree models in browse mode now share the first 10 column types
containing all the necessary information to display the model on screen.
Therefor it is now easy to just operate on the tree model associated
with the file tree view and in most cases it isn't necessary anymore to
special case the browse modes.

Simplify gtk_file_chooser_default_unselect_file()

Creating the model no longer fails

So remove the code that checked for failure, as it's not needed anymore.

Implement new GtkFileSystemModel

The new model is mostly API-compatible with the old model (minimal
changes were required), but is a lot faster and has a lot of very
desirable features.
- the model does no longer support a tree, just a list of files in a
  given directory
- the storage has been moved to a GArray as opposed to a tree
- no more dependency on GtkFileSystem
- columns are managed by the creator of the model, so any number of
  nodes can be added as needed. This also makes the API more similar
  to GtkListStore.
- Values are filled on demand using a function given when creating the
  model.
- The function can decide to let the model cache returned values or
  decide to be called again the next time the value is queried.
- implements GtkTreeSortable
- _gtk_file_system_model_get_value() was added to significantly speed
  up value access, which is necessary when sorting large models.

move GtkFileSystemModel private stuff out of the private header

Do not assign the parent class twice

G_DEFINE_TYPE already does this for us, no need to do it again in
class_init

https://bugzilla.gnome.org/show_bug.cgi?id=598515

Notify the "active" property of the spinner only when needed

Notify spinner::active only if it really changed an other cosmetic fixes

Updated Slovenian translation

Updated Slovenian translation

gtk-demo: Remove C99-ism from list_store example

Updated Swedish translation

Completing translation for Punjabi

Move documentation to inline comments: GtkToolShell

https://bugzilla.gnome.org/show_bug.cgi?id=597865

Move documentation from templates to inline comments: GtkAccesible

https://bugzilla.gnome.org/show_bug.cgi?id=597865

Move documentation from templates to inline comments: GtkActivatable

https://bugzilla.gnome.org/show_bug.cgi?id=597865

Move documentation from templates to inline comments: GtkAccelGroup

https://bugzilla.gnome.org/show_bug.cgi?id=597865

Fix GtkSpinner using style before it's set

Fix a possible division by zero when the spinner
is started before it's realized (eg. before it has a style
set).

https://bugzilla.gnome.org/show_bug.cgi?id=598496

Fix property links in GtkSpinner

Updated Russian translation

Add GtkSpinner::animation-duration style property

Add GtkSpinner to the widget gallery

Yes, our documentation setup is wierd and wonderful.

Documentation and stylistic fixups

Fix "active" property not being notified

And setting it FALSE starting the spinning when it should
stop it.

Fix doc build

There was an escaping problem in the newly inlined documentation.

Capitalise a11y description for GtkSpinner

Update POTFILES.in for spinner widgets

Bug 319607 – Add a throbber (activity widget) to GTK+

Add GtkSpinner activity throbber, as well as a cell renderer.

Add ability to set a tooltip's image from a GIcon

Makes it easier to implement fallbacks whilst following
theme changes.

https://bugzilla.gnome.org/show_bug.cgi?id=598261

Move documentation from templates to inline comments: GtkAboutDialog

https://bugzilla.gnome.org/show_bug.cgi?id=597865

Substitute gtk_widget_ref/unref with g_object_ref/unref

Substitute the use of gtk_widget_ref/unref in documentation examples
and in internal gtk+ code

https://bugzilla.gnome.org/show_bug.cgi?id=598218

Substitute gtk_style_unref for g_object_unref in documentation example

https://bugzilla.gnome.org/show_bug.cgi?id=598218

Use g_object_ref/unref instead deprecated gdk_*_ref/unref functions

Substitute deprecated reference counting functions for
g_object_ref/unref in documentation and in internal code

https://bugzilla.gnome.org/show_bug.cgi?id=598217

Don't use XP themes until they work

Bump version to 2.19.0

Updated Russian translation

Updated Russian translation

po/vi.po: update translation of "Places"

Updated Catalan translation

Updated Catalan translation

Bug 596019 - No accessors for GtkDialog buttons

Add API for GtkDialog to return widgets by response ID.
Added gtk_dialog_get_widget_for_response() to access to all kinds
of buttons with all kinds of responses.

Updated Romanian translation